Skip to content

IRGraph C++ Printer.#9042

Open
mcourteaux wants to merge 13 commits intohalide:mainfrom
mcourteaux:ircppprint
Open

IRGraph C++ Printer.#9042
mcourteaux wants to merge 13 commits intohalide:mainfrom
mcourteaux:ircppprint

Conversation

@mcourteaux
Copy link
Copy Markdown
Contributor

@mcourteaux mcourteaux commented Mar 15, 2026

Should be useful for constructing IR trees and graphs for debugging, constructing tests, especially from generated expressions in failing fuzz tests.

Currently focused on printing IR Exprs, not Stmts.

@mcourteaux
Copy link
Copy Markdown
Contributor Author

Failure is: #9055 (comment)

@mcourteaux mcourteaux requested a review from abadams March 23, 2026 19:03
@mcourteaux
Copy link
Copy Markdown
Contributor Author

mcourteaux commented Mar 27, 2026

Move it to test/common. Nope: move it to test/fuzz

Copy link
Copy Markdown
Member

@alexreinking alexreinking left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with nit

target_compile_definitions(Halide_fuzz PUBLIC HALIDE_FUZZER_BACKEND=HALIDE_FUZZER_BACKEND_LIBFUZZER)
endif ()


Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fuzz simplify complained for me that HALIDE_FUZZER_BACKEND is undefined. It's no longer the INTERFACE library, but an OBJECT library, so I think it's normal you have to declare it as a PUBLIC compile definition?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, the extra blank line

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah, didn't realize. precommit didn't complain :P

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants